Skip to content

enhance: do not load huge diffs by default - #2869

Open
bertyhell wants to merge 1 commit into
sourcegit-scm:developfrom
bertyhell:perf/large-diff-guard
Open

bertyhell wants to merge 1 commit into
sourcegit-scm:developfrom
bertyhell:perf/large-diff-guard

Conversation

@bertyhell

@bertyhell bertyhell commented Nov 9, 2026 •

Copy link
Copy Markdown
image

Reading and parsing the whole output of git diff for a huge generated file (lock file,
minified bundle, data dump, ...) can freeze the diff view for a long time and use a lot of
memory. Stop reading once the diff output exceeds 5 MB, kill the git diff process and show
a placeholder instead. The SHOW ANYWAY button reloads the diff of that file without the
limit (only for the currently selected file).

🤖 Generated with Claude Code

Reading and parsing the whole output of `git diff` for a huge generated file (lock file,
minified bundle, data dump, ...) can freeze the diff view for a long time and use a lot of
memory. Stop reading once the diff output exceeds 5 MB, kill the `git diff` process and show
a placeholder instead. The `SHOW ANYWAY` button reloads the diff of that file without the
limit (only for the currently selected file).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KCxH1zwDXLd9kfAAkcf6PZ

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants